Skip to content

chore(ci): fix codecov upload commit SHA and coverage source paths#1370

Merged
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/fix-codecov
Feb 26, 2026
Merged

chore(ci): fix codecov upload commit SHA and coverage source paths#1370
hkad98 merged 1 commit intogooddata:masterfrom
hkad98:jkd/fix-codecov

Conversation

@hkad98
Copy link
Contributor

@hkad98 hkad98 commented Feb 26, 2026

Override commit SHA to use PR head instead of merge commit, pass PR number explicitly, and strip Docker /data/ prefix from coverage XML source paths so Codecov can match files to the repo.

JIRA: trivial
risk: nonprod

@codecov
Copy link

codecov bot commented Feb 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.60%. Comparing base (c00ca3b) to head (123e8e7).
⚠️ Report is 21 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1370       +/-   ##
===========================================
+ Coverage    0.00%   77.60%   +77.60%     
===========================================
  Files         158      224       +66     
  Lines       11048    14466     +3418     
===========================================
+ Hits            0    11227    +11227     
+ Misses      11048     3239     -7809     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hkad98 hkad98 force-pushed the jkd/fix-codecov branch 2 times, most recently from 5a3778f to 4fed30f Compare February 26, 2026 14:39
Override commit SHA to use PR head instead of merge commit and pass PR
number explicitly. Add coverage path mapping to all packages so
coverage.py maps installed site-packages back to src/ (needed because
tox uses wheel packaging).

JIRA: trivial
risk: nonprod
@hkad98 hkad98 merged commit 3698ce1 into gooddata:master Feb 26, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants